Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix sticky scroll start line number #157466

Merged

Conversation

DingWeizhe
Copy link
Contributor

@DingWeizhe DingWeizhe commented Aug 8, 2022

Fixes #157680
Fixes #157180
Fixes #157293
Fixes #157380
Fixes #157375


fix #157375

when class or method have decorator(TypeScript) or attribute(C#),
range.startLineNumber is begin of decorator, but selectionRange.startLineNumber is start of method.

image

@gjsjohnmurray
Copy link
Contributor

/assign @aiday-mar

@DingWeizhe DingWeizhe force-pushed the fix-sitkcy-scroll-start-line-number branch from 3d7babc to 3388612 Compare August 9, 2022 08:07
@DingWeizhe
Copy link
Contributor Author

#157483 be merged with lots of changes so I rebase the PR

@aiday-mar
Copy link
Contributor

aiday-mar commented Aug 10, 2022

@DingWeizhe Thank you for your contribution, this will be merged into the existing code base

joyceerhl pushed a commit that referenced this pull request Aug 10, 2022
Co-authored-by: dweizhe <d.weizhe@anyong.com.tw>
@github-actions github-actions bot locked and limited conversation to collaborators Sep 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.